Skip to content

Use updatedAt for query param sorting - #409

Merged
brucetony merged 1 commit into
developfrom
refactor/hub-camel-case
Aug 8, 2026
Merged

Use updatedAt for query param sorting#409
brucetony merged 1 commit into
developfrom
refactor/hub-camel-case

Conversation

@brucetony

@brucetony brucetony commented Aug 8, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Bug Fixes
    • Updated project and analysis listings to sort by the correct “Last updated” field.
    • Ensured project nodes, projects, and analyses consistently display the most recently updated items first.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9cbe85aa-0aac-45a8-8c0a-a52806eb124c

📥 Commits

Reviewing files that changed from the base of the PR and between c285322 and 1bacfa0.

📒 Files selected for processing (3)
  • app/components/analysis/AnalysesTable.vue
  • app/composables/useAPIFetch.ts
  • app/composables/useProjectAnalysisSummary.ts
📝 Walkthrough

Walkthrough

The change replaces legacy updated_at sort parameters with updatedAt across shared API fetches, the analyses table, and the project analysis summary. One import is reformatted without changing behavior or exported declarations.

Changes

API sort field update

Layer / File(s) Summary
Update shared API sorting
app/composables/useAPIFetch.ts		 Project-node, project, and analysis-node requests now sort by -updatedAt.
Update component and summary queries
app/components/analysis/AnalysesTable.vue, app/composables/useProjectAnalysisSummary.ts		 Project and analysis-node queries now use updatedAt sorting. The API import is reformatted without behavior changes.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: using updatedAt for query parameter sorting.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/hub-camel-case

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@brucetony
brucetony merged commit 98ad381 into develop Aug 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@brucetony